Skip to content

chore(release): check mapContent file field#464

Merged
e11sy merged 1 commit intomasterfrom
imp/check-original-filename
Oct 5, 2025
Merged

chore(release): check mapContent file field#464
e11sy merged 1 commit intomasterfrom
imp/check-original-filename

Conversation

@e11sy
Copy link
Copy Markdown
Member

@e11sy e11sy commented Oct 5, 2025

Problem

according to source-map 3 spec
File: An optional name of the generated code that this source map is associated with.
And we rely on an optional field

Solution

if mapContent.file is not defined — use stripped version of the map filename
e.g. for bundle.js.map we would have bundle.js as an originFileName

@e11sy e11sy merged commit 1de0465 into master Oct 5, 2025
4 checks passed
@e11sy e11sy deleted the imp/check-original-filename branch October 5, 2025 21:02
e11sy added a commit that referenced this pull request Oct 5, 2025
* chore: added try-catch for sending report (#457)

* chore: added try-catch for sending report

* added res log

* chore(license): adopt BSL 1.1 (#460)

* chore: update license to BUSL-1.1 in package.json and README.md

* chore: update licensed work reference in LICENSE file

* imp(): get rid of duplicated releases (#462)

* imp(): get rid of duplicated releases

* chore(): eslint fix

* fix(grouper): skip notifications for muted errors (#459)

* feat(grouper): add event ignored check before notifying worker

* Implemented a new method to check if an event is marked as ignored.
* Updated the notifier task addition to only proceed if the event is not ignored.

* Removed the isEventIgnored method as its functionality is now integrated directly into the task addition logic.

* Removed unnecessary line break

* update @hawk.so/types dependency to version 0.1.35

* add tests for event marks handling in NotifierWorker

* fix

* chore(release): check mapContent file field (#464)

---------

Co-authored-by: Vyacheslav Chernyshev <81693471+slaveeks@users.noreply.github.com>
Co-authored-by: Dobrunia Kostrigin <48620984+Dobrunia@users.noreply.github.com>
Co-authored-by: e11sy <130844513+e11sy@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants